Skip to content

better json schema errors#178

Closed
pelikhan wants to merge 13 commits intogithub:mainfrom
githubnext:main
Closed

better json schema errors#178
pelikhan wants to merge 13 commits intogithub:mainfrom
githubnext:main

Conversation

@pelikhan
Copy link
Contributor

Doing some magic to locate errors from json schema back into the source.

Copilot AI and others added 12 commits August 21, 2025 03:55
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
- Add internal/mapper package with MapErrorToSpans API
- RFC6901 JSON Pointer decoding with proper escaping
- goccy/go-yaml AST traversal with position metadata
- Error-kind specific mapping (type, required, additionalProperties)
- Fallback heuristics for missing properties and out-of-range indices
- Confidence scoring system (0.2-1.0 range)
- Comprehensive test suite with 100% coverage
- CLI test runner for interactive debugging
- Full documentation with algorithm details

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
…e locations

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
…n.go

- Extract all JSON schema validation related functions to dedicated file
- Move httpURLLoader, validateWorkflowSchema*, formatSchemaValidationError, extractErrorMeta, formatValidationMessage, generateValidationHint, and extractContextLines
- Remove unused imports from compiler.go (net/http, jsonschema, mapper)
- Maintain all existing functionality and method signatures
- All tests continue to pass with the refactored code structure

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
…af62-a58980a2e9e1

Add draft boolean option to output.pull-request feature
…g integration

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
…b73e-fe64be5b3b8a

Add comprehensive JSON Schema validation test suite with error mapping integration
@pelikhan pelikhan closed this Aug 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants